-
Notifications
You must be signed in to change notification settings - Fork 169
#704: Bumped to SpringBoot4, SpringFramework7 and JDK17 #705
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Are you sure you want to require Java 25? Seems a bit high to me. Spring Boot 4/Framework 7 require Java 17, the same with Jackson 3. I personally would have no issues with 25, but the broader community might not be ready yet. |
|
You're right, probably too excessive. I've downgraded it to 17. Jackson2 is deprecated in Spring Framework 7, it also causes a lot of extra headache if you try to run this in a project that is Spring Framework 7 and having to either be stuck using Jackson2 or shoehorn in both versions of the dep. |
Signed-off-by: David Strömner <david.stromner@stralfors.se>
Signed-off-by: David Strömner <david.stromner@stralfors.se>
Signed-off-by: David Strömner <david.stromner@stralfors.se>
Signed-off-by: David Strömner <david.stromner@stralfors.se>
Signed-off-by: David Strömner <david.stromner@stralfors.se>
Signed-off-by: David Strömner <david.stromner@stralfors.se>
a1e3fca to
cfd9af3
Compare
Signed-off-by: David Strömner <david.stromner@stralfors.se>
Signed-off-by: David Strömner <david.stromner@stralfors.se>
|
Ready for review again |
|
Hello, any thoughts when a new version with these changes will be released? |
Do you want a review from someone in particular, or a random guy from the Internet would also be OK? I might find some time this or next week, but I cannot promise anything. Still, I'm blocked with upgrades because of |
I always welcome extra eyes on code I've written so by all means if it interest you! Otherwise it was a maintainer I'm waiting for so it might get approved some day |
Resolves issue #704